Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: fix edge-gateway build #223

Merged
merged 9 commits into from
Dec 3, 2024
Merged

feat: fix edge-gateway build #223

merged 9 commits into from
Dec 3, 2024

Conversation

travis
Copy link
Collaborator

@travis travis commented Dec 3, 2024

upgrade dependencies and adjust process to match the reads/packages/edge-gateway build:

https://github.com/storacha/reads/blob/main/.github/workflows/edge-gateway.yml

upgrade dependencies and adjust process to match the `reads/packages/edge-gateway` build:

https://github.com/storacha/reads/blob/main/.github/workflows/edge-gateway.yml
Copy link

cloudflare-workers-and-pages bot commented Dec 3, 2024

Deploying nftstorage-link with  Cloudflare Pages  Cloudflare Pages

Latest commit: 161213d
Status:🚫  Build failed.

View logs

@travis
Copy link
Collaborator Author

travis commented Dec 3, 2024

I had to do a bit more work here to get this building and am leaving a few things un-done:

  1. the website cloudflare build is failing
  2. the API tests are failing because docker seems not to be starting up as they expect
  3. the client tests are failing because of a JavaScript error

in all cases I don't think my changes cause the issue, and it's not clear the projects that are failing will ever be deployed again, so I'm punting fixing them to whenever we know we need to get them deployable again - happy for that to be me soon if we all agree, but I'd like to get this logging issue straightened out asap!

@travis travis merged commit 484ff3a into main Dec 3, 2024
12 of 18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants